Skip to content

Conversation

@dnhatn
Copy link
Member

@dnhatn dnhatn commented Aug 27, 2025

If the target shard is empty or does not match the query, processedShards will be empty.
This change relaxes the assertion to permit these cases.

Closes #133624

@dnhatn dnhatn added >test Issues or PRs that are addressing/adding tests :StorageEngine/TSDB You know, for Metrics labels Aug 27, 2025
@dnhatn dnhatn marked this pull request as ready for review August 27, 2025 16:43
@dnhatn dnhatn added the auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) label Aug 27, 2025
@dnhatn dnhatn enabled auto-merge (squash) August 27, 2025 16:43
@elasticsearchmachine
Copy link
Collaborator

Pinging @elastic/es-storage-engine (Team:StorageEngine)

@dnhatn dnhatn disabled auto-merge August 27, 2025 20:44
@dnhatn dnhatn merged commit b84a475 into elastic:main Aug 27, 2025
32 of 33 checks passed
@dnhatn dnhatn deleted the fix-profile branch August 27, 2025 20:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) :StorageEngine/TSDB You know, for Metrics Team:StorageEngine >test Issues or PRs that are addressing/adding tests v9.2.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[CI] TimeSeriesIT testProfile failing

3 participants